Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve test support #806

Merged
merged 3 commits into from
Oct 17, 2023
Merged

Improve test support #806

merged 3 commits into from
Oct 17, 2023

Conversation

ktf
Copy link
Member

@ktf ktf commented Oct 17, 2023

Publish coverage information also from envsitedir

This is needed because otherwise we do not publish the
coverage information of the unit tests, but only those
coming from the aliBuild commands.

@TimoWilken
Copy link
Contributor

According to the tox docs, you should be able to use {env:VARIABLE_NAME} in commands to substitute env vars.

@TimoWilken
Copy link
Contributor

Ah, you changed it already! :)

@ktf ktf force-pushed the pr806 branch 12 times, most recently from e4d9f15 to 75f7295 Compare October 17, 2023 12:21
@ktf ktf changed the title Fix macOS detection when running standalone Dump test information Oct 17, 2023
@ktf ktf changed the title Dump test information Always import platform Oct 17, 2023
@ktf ktf changed the title Always import platform Attempts to publish coverage invo Oct 17, 2023
@ktf ktf force-pushed the pr806 branch 2 times, most recently from 9a556d3 to 7316f53 Compare October 17, 2023 13:10
@ktf ktf force-pushed the pr806 branch 3 times, most recently from baaef53 to de6ff60 Compare October 17, 2023 13:25
@ktf ktf changed the title Attempts to publish coverage invo Publish coverage information also from envsitedir Oct 17, 2023
@ktf ktf force-pushed the pr806 branch 2 times, most recently from 7322262 to 57f76ab Compare October 17, 2023 13:47
Long gone are the days this was not available
This is needed because otherwise we do not publish the
coverage information of the unit tests, but only those
coming from the aliBuild commands.
@ktf ktf changed the title Publish coverage information also from envsitedir Improve test support Oct 17, 2023
@ktf ktf merged commit 522effc into alisw:master Oct 17, 2023
20 checks passed
@ktf ktf deleted the pr806 branch October 17, 2023 14:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants